Skip to content

Conversation

@emersion
Copy link
Member

@emersion emersion commented Oct 7, 2025

See individual commits.

Closes: #12395

@github-actions github-actions bot added the area:front Work on Standard OSRD Interface modules label Oct 7, 2025
@emersion
Copy link
Member Author

emersion commented Oct 7, 2025

Waiting a bit until rollup/plugins#1905 gets merged.

@emersion
Copy link
Member Author

I've tried adding a dependency version override as described in rollup/plugins#1708 but the rollup plugin still ended up grabbing ESLint v8 instead of v9 for some reason…

@achrafmohye
Copy link
Contributor

LGTM :) everything builds, runs, and behaves as expected thanks @emersion

Copy link
Contributor

@clarani clarani left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM ✅

The right-hand-side of the expression is never used, because a
non-empty string is always truthy.

Signed-off-by: Simon Ser <[email protected]>
Unused eslint-disable comments now cause ESLint warnings.

Signed-off-by: Simon Ser <[email protected]>
Signed-off-by: Simon Ser <[email protected]>
Closes: #12395
@emersion emersion enabled auto-merge November 4, 2025 12:31
@emersion emersion added this pull request to the merge queue Nov 4, 2025
Merged via the queue into dev with commit 5888d31 Nov 4, 2025
29 checks passed
@emersion emersion deleted the eslint-v9 branch November 4, 2025 12:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:front Work on Standard OSRD Interface modules

Projects

None yet

Development

Successfully merging this pull request may close these issues.

front: upgrade ESLint to v9

4 participants